Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Revert options & yargs parser changes #257

Merged
merged 1 commit into from
Mar 10, 2024
Merged

Conversation

phated
Copy link
Member

@phated phated commented Mar 10, 2024

This moves the yargs parser logic back into the index.js file. I want to try cleaning up the CLI help in my translations prototype and the indirection of throwing an error was confusing for me.

@phated phated requested a review from sttk March 10, 2024 22:52
@phated phated force-pushed the phated/cli-options branch from cbc3572 to 93b81d3 Compare March 10, 2024 22:54
@phated phated force-pushed the phated/cli-options branch from 93b81d3 to 788bfde Compare March 10, 2024 22:57
@phated phated merged commit 4fc66f6 into master Mar 10, 2024
17 checks passed
@phated phated deleted the phated/cli-options branch March 10, 2024 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant